I rearranged the accounts elsewhere listing, per #2052 (comment).

The "One-off Receiving Options" changes to "One-off Giving Options" when viewing someone else's profile.

I'm seeing serious brokenness (along the lines of #2052 (comment)) for the group elsewhere pages (e.g.):

This says "Balanced" right now, but will say Gittip in the future.

Per IRC, Balanced will control a Coinbase OAuth app in our name. They'll store the secret.

We added a little note about the new feature, but you may or may not want that.

Right, thanks. I think we'll take that out. :-)

Per IRC, Balanced will control a Coinbase OAuth app in our name. They'll store the secret.

Correct. The idea is that we manage all the shenanigans of connecting to Coinbase for you, it's the same as tokenizing a card. In this case, the OAuth secret is like the card number.

I'm seeing the "Edit" buttons replaced with links:

vs.

Does that belong in this PR?

/cc @kyungmin , though ultimately this is a product decision, really.

@whit537 The latest commit should fix the unintentional button size changes. I agree that the "edit" change is irrelevant to this PR.

Found visual styling defect, but reported in #2065 since it's tangential

Thanks @kyungmin. I'm still seeing problems related to #2053 (comment), however.

We've deployed some of the Connected Accounts style changes. Here's a screenshot:

It's time to start chipping away at the Coinbase integration itself if we can. The most significant change involved here is that we are moving from one way to pay in, to two ways, which is basically infinite ways according to the 0, 1, infinity rule. The approach I'm seeing on this PR is to try charging a credit card, and then fall back to charging an "external_account":

            try:
                source = customer.cards.one()
            except balanced.exc.wac.NoResultFound:
                source = customer.external_accounts.one()

What concerns me about this approach is that Gittip's funding flow already confuses our users, and adding this implicit rule about cards and external accounts is only going to further muddy the water. See:

• credit card statement should only be for the difference #1486, where a user who is both giving and receiving is confused about the amount they are being charged.
• Fix for issue #757 #771, where people think that by adding a bank account, they'll be able to use that to add funds (ACH debit; we only use bank accounts for ACH credit right now).
• Similar confusion exists around Venmo and Bitcoin—We allow connecting these funding accounts but don't actually use them anywhere.

I suppose that the easiest way to address this confusion without losing ourselves down a rabbit hole (#1063) is the one we've already identified: to group accounts in the UI and label them appropriately. That should be our next PR.

After that we can bust up the monolithic "Balanced Payments" item in the connected accounts listing into two: Credit Card (under "Adding Money"), and Bank Account (under "Withdrawing Money").

At that point we can look at adding Coinbase as an additional item under Adding Money.

@ESWAT Wanna take a crack at grouping/labeling the Connected Accounts? Here's what I think we want:

• Moving Money Every Week - Balanced Payments
• Other {Giving,Receiving} Options - Bitcoin, Venmo
• Social Profiles—Bitbucket, Bountysource, GitHub, OpenStreetMap, Twitter

I took a pass at this in #2124 so you can scavenge for parts there if you like. Iirc you'll want to change <table id="accounts"> to <table class="accounts"> and have several of them. You can access website.platforms.venmo and skip that one in the loop for Social Profiles, or whatevs. Also, "Other Giving Options" should be "Other Receiving Options" when a member is viewing their own profile.

@whit537 Sure thing. Is the grouping task higher priority over restyling the account avatars we’ve talked about in #2103 (comment)?

Is the grouping task higher priority over restyling the account avatars we’ve talked about

@ESWAT Yes, thanks for asking. This #2053 PR is the biggest thing on our plate right now. We need to carve up this elephant and eat it all ASAP.

Cool. Going to create a new Issue to deal with the grouping then.

thx :)

Hey, would someone mind explaining what parts of this still need to be pulled out into separate PRs, just so everyone has some idea of how it's going?

Thanks!

This weekend I was planning on working on using CC holds to not hit people's cards when they'd otherwise have enough money incoming, but Coinbase really is more important. Towards that end, the first thing I'll be working on is #1063. Time permitting, I'll also work on Coinbase integration itself. If anyone has time this weekend to pair on this, please let me know.

IRC

From above:

After that we can bust up the monolithic "Balanced Payments" item in the connected accounts listing into two: Credit Card (under "Adding Money"), and Bank Account (under "Withdrawing Money").

At that point we can look at adding Coinbase as an additional item under Adding Money.

I can work on splitting the current Balanced Payments section to the one in coinbase, to make room for the coinbase account being displayed.

Now that the adding/withdrawing money sections are split up (#2297), let's look at adding Coinbase as an additional way to add money. If we don't get to #1063 first we should do like @kyungmin & @matthewfl did and simply try Coinbase if the CC doesn't work.

Sounds like @Aaron1011 might take this on. IRC

This is obsolete, closing in favor of #2841.